Skip to content

kungfub3s: replace BAD_DUMP sound ROM with one from another romset#15337

Merged
happppp merged 5 commits into
mamedev:masterfrom
clawgrip:patch-5
May 19, 2026
Merged

kungfub3s: replace BAD_DUMP sound ROM with one from another romset#15337
happppp merged 5 commits into
mamedev:masterfrom
clawgrip:patch-5

Conversation

@clawgrip
Copy link
Copy Markdown
Contributor

Since there's no useful information at all

Comment thread src/mame/irem/m62.cpp Outdated

ROM_REGION( 0x10000, "irem_audio:iremsound", 0 )
ROM_LOAD( "3.bin", 0xa000, 0x2000, BAD_DUMP CRC(b4293435) SHA1(5e2b96c19c4f5c63a5afa2de504d29fe64a4c908) ) // Empty ROM
ROM_LOAD( "3.bin", 0xa000, 0x2000, NO_DUMP )
Copy link
Copy Markdown
Member

@angelosa angelosa May 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs ROMREGION_ERASEFF to preserve current behaviour (the ROM is 1-filled).

Assuming we even want to do this, looks a YAGNI to me?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm just trying to avoid false positives when looking for new dumps matches...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ROMREGION_ERASEFF fills not just the part where the dead ROM was, but also the unused part below 0xa000 with 0xff. Not sure if that’s desired. Does 0xff vs 0x00 in the dead ROM space really improve anything for the 6803?

@happppp
Copy link
Copy Markdown
Member

happppp commented May 18, 2026

May I suggest this instead:

ROM_LOAD( "3.bin", 0xa000, 0x2000, BAD_DUMP CRC(58e87ab0) SHA1(3b03c101fec58eac13fc309a78df9a2cd44f7604) ) // Taken from other set, probably correct

@clawgrip
Copy link
Copy Markdown
Contributor Author

May I suggest this instead:

ROM_LOAD( "3.bin", 0xa000, 0x2000, BAD_DUMP CRC(58e87ab0) SHA1(3b03c101fec58eac13fc309a78df9a2cd44f7604) ) // Taken from other set, probably correct

Much better, thanks a lot

@happppp happppp changed the title irem/m62.cpp: An empty dump is a NO_DUMP, not a BAD_DUMP kungfub3s: replace BAD_DUMP sound ROM with one from another romset May 19, 2026
@happppp happppp merged commit dda3ab8 into mamedev:master May 19, 2026
6 checks passed
@clawgrip clawgrip deleted the patch-5 branch May 19, 2026 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants